Conversation
🦋 Changeset detectedLatest commit: 1667dc5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
|
@paoloricciuti I added skills for opencode & claude, but not for others. I know that cursor is looking at skills as well (maybe even in .claude/skills). Could you advise? |
|
@jycouet I think almost every tool nowadays looks at skills, you can check the right folder with the skills.sh cli |
|
I wonder if we should just invoke that actually 🤔 |
Since it's solving only |
We leave it like that for now? :D |
# Conflicts: # packages/sv/src/addons/index.ts
- delivery option: Svelte plugin (Claude via committed .claude/settings.json, opencode via opencode.json) or individual tools - granular MCP/skills/sub-agents selection; rename `setup` -> `mcpSetup` - consolidate per-client config into a single `CLIENTS` registry
manuel3108
left a comment
manuel3108
left a comment
There was a problem hiding this comment.
LGTM from a code standpoint. I would need a claude skill to be able to review this from a content perspective, but I will leave that to paolo.
Still not a huge fan of having all of those md files in our repo, but that's fine for the moment to finally get this PR merged.
(I havnt tested anything)
| import { defineAddon, defineAddonOptions } from '../core/config.ts'; | ||
| import { getSharedFiles } from '../create/utils.ts'; | ||
|
|
||
| const RX_MD = /\.md$/; |
There was a problem hiding this comment.
Had to think second.
| const RX_MD = /\.md$/; | |
| const REGEX_MD = /\.md$/; |
3 participants
Closes #1044
Description
replace
mcpadd-on withai-toolsadd-on that includes both MCP and skills setupChecklist